-
-
Notifications
You must be signed in to change notification settings - Fork 272
GLM-robust update #451
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
GLM-robust update #451
Conversation
update fork
Rebasing
* Fix start and end of the files * Fix start and end for ipynb * Fix start and end for ipynb * Fix start and end for ipynb * Fix notebook name and watermark * Pass pre-commit * Add authorship * Pre-commit checks Co-authored-by: Alihan Zihna <a.zihna@ckhgbdp.onmicrosoft.com>
* Bring stochastic volatility notebook in line with style guide Fixing dashes in reference entry Removing extra lines from reference bib file * Adding original author information to stochastic volatility notebook * Adding updated myst file * Fixing misspecified tags and updating author list * Update stochastic_volatility.myst.md
* ⬆️ upgrade nb to V4 Co-authored-by: Ana Rita Santos <ar.oliveirasantos@gmail.com> * 👌 updates due to review comments * 👌 sampling with multivariate_normal from xarray_einstats dev * 📌 set ablog version * 🍻 🐛 fix the pinning * 📌 pin sphinx and sphinx gallery * 📝 add watermark header Co-authored-by: Ana Rita Santos <ar.oliveirasantos@gmail.com>
* notebook: divergences; update notebook formatting * notebook divergence: updates,figure metadata * Divergence nb: run notebook * make code across one line * make code one line to fix myst errors from precommit * rm space around = sign * nb divergence: fix myst error with spacing * make Authorship section consistent betw notebook and myst * remove myst file * divergence notebook, after running pre-commit * run myst for divergence nb * added RS authorship to end * rerun myst on divergence nb * divergence nb: rerun myst
Re-wrote "How to debug a model" (pymc-devs#406)
Reinforcement Learning Notebook (pymc-devs#410)
Catching up with main
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
@OriolAbril @reshamas - it looks like this change hasn't been reviewed yet (I filed this PR 16 days ago). clearly no rush with this, but I want to make sure it doesn't fall through the cracks. Thanks! |
We currently have 3 PRs open for this same notebook: #356, #445, #451. And I am not sure how to go about this, git conflicts on notebooks are a nightmare. I might try to merge the myst and then update the notebook from it which might happen in a 4th PR overriding the current 3. I will take a look and work on this next week. Thank you all for the PRs, I'll make sure at the very least you are listed as co-author in the merged PR. |
@OriolAbril - ouch, I see! That's unfortunate, my apologies that it took me so long to finish with it! |
@kuvychko |
Superseded by #472 |
@reshamas - sure thing, igor.kuvychko@gmail.com Thanks!! |
Tracker Issue #88
Summary of changes:
rv.logp(x)
method topm.logp(rv, x)
since the original method was removed